Switch to using papilo develop branch#493
Switch to using papilo develop branch#493rapids-bot[bot] merged 1 commit intoNVIDIA:branch-25.10from
Conversation
There was a problem hiding this comment.
Greptile Overview
Greptile Summary
This PR updates the papilo dependency in cuOPT's C++ build configuration to switch from a temporary bug fix branch to the main develop branch. The change modifies cpp/CMakeLists.txt to update the git commit hash from 8f710e33d352bf319d30b9c57e70516222f3f5ca (previously tracking a "cliquemergebug" branch) to 741a2b9c8155b249d6df574d758b4d97d4417520 (now tracking the develop branch as of Oct 12, 2025).
The reason for this change is that the bug fixes for the cliquemerging presolver, which were previously isolated in a separate branch, have now been merged into papilo's develop branch. This represents a natural progression in dependency management - moving from a temporary fix back to the main development stream once fixes are integrated. The updated comments in the CMakeLists.txt file now reflect that cuOPT is tracking the develop branch rather than a specific bug fix branch, maintaining transparency about why the develop branch is used instead of the main branch (due to needed presolvers and settings not yet in main).
Important Files Changed
Changed Files
| Filename | Score | Overview |
|---|---|---|
| cpp/CMakeLists.txt | 4/5 | Updated papilo dependency from bug fix branch to develop branch with new commit hash |
Confidence score: 4/5
- This PR is safe to merge with minimal risk as it's a straightforward dependency update
- Score reflects the low-risk nature of updating to a develop branch that contains previously tested bug fixes, though not a full 5 due to the inherent risk of tracking a development branch
- Pay close attention to ensuring the new commit hash contains all previously required functionality from the bug fix branch
1 file reviewed, 1 comment
|
/ok to test 871f4a3 |
|
/merge |
Description
We were using a temporary bug fix branch that has bug fixes for cliquemerging presolver. This branch is now merged into develop branch. So we are switching to develop branch now.
Issue
Checklist